Skip to content

Update to Gradle 9.2.1#20391

Merged
cwperks merged 1 commit into
opensearch-project:mainfrom
andrross:gradle-9.2.1
Jan 9, 2026
Merged

Update to Gradle 9.2.1#20391
cwperks merged 1 commit into
opensearch-project:mainfrom
andrross:gradle-9.2.1

Conversation

@andrross
Copy link
Copy Markdown
Member

@andrross andrross commented Jan 8, 2026 •
edited by coderabbitai Bot
Loading

Check List

  • Functionality includes testing.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Summary by CodeRabbit

  • Chores
    • Updated build tooling to the latest patch version for improved stability and reliability.

✏️ Tip: You can customize this high-level summary in your review settings.

@andrross
Update to Gradle 9.2.1
8aeb48f 
Signed-off-by: Andrew Ross <andrross@amazon.com>
@andrross andrross requested a review from a team as a code owner January 8, 2026 19:16
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Jan 8, 2026 •
edited
Loading

📝 Walkthrough

Walkthrough

Gradle wrapper version bumped from 9.2.0 to 9.2.1 in the wrapper properties file with corresponding SHA-256 checksum update. No code logic or control flow changes.

Changes

Cohort / File(s) Summary
Gradle Wrapper Version Update
gradle/wrapper/gradle-wrapper.properties		 Version bumped from gradle-9.2.0-all.zip to gradle-9.2.1-all.zip; SHA-256 checksum updated from 16f2b958... to f8634427... for validation alignment

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested labels

dependencies, patch

Suggested reviewers

  • cwperks

Poem

🐰 A tiny hop from 9.2.0 to point-one,
The checksum aligns beneath the sun,
Gradle's wrapper spins with grace so true,
A patch update—fresh and new! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description check ⚠️ Warning The description is largely incomplete, missing the required 'Description' and 'Related Issues' sections from the template; only the checklist and license confirmation are provided. Add a 'Description' section explaining what the update achieves and include a 'Related Issues' section, even if no issues are resolved.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Update to Gradle 9.2.1' directly and clearly summarizes the main change - updating the Gradle wrapper to version 9.2.1.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 733931b and 8aeb48f.

📒 Files selected for processing (1)
  • gradle/wrapper/gradle-wrapper.properties
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (20)
  • GitHub Check: gradle-check
  • GitHub Check: assemble (25, windows-latest)
  • GitHub Check: assemble (25, ubuntu-24.04-arm)
  • GitHub Check: assemble (25, ubuntu-latest)
  • GitHub Check: assemble (21, ubuntu-24.04-arm)
  • GitHub Check: assemble (21, windows-latest)
  • GitHub Check: assemble (21, ubuntu-latest)
  • GitHub Check: Analyze (java)
  • GitHub Check: precommit (25, macos-15)
  • GitHub Check: precommit (21, windows-2025, true)
  • GitHub Check: precommit (21, ubuntu-latest)
  • GitHub Check: precommit (21, macos-15)
  • GitHub Check: precommit (25, windows-latest)
  • GitHub Check: precommit (25, macos-15-intel)
  • GitHub Check: precommit (25, ubuntu-24.04-arm)
  • GitHub Check: precommit (25, ubuntu-latest)
  • GitHub Check: precommit (21, macos-15-intel)
  • GitHub Check: precommit (21, ubuntu-24.04-arm)
  • GitHub Check: precommit (21, windows-latest)
  • GitHub Check: detect-breaking-change
🔇 Additional comments (1)
gradle/wrapper/gradle-wrapper.properties (1)

3-3: Verify the SHA-256 checksum and Gradle 9.2.1 compatibility.

The wrapper update from Gradle 9.2.0 to 9.2.1 looks structurally sound (HTTPS URL, checksum validation enabled), but the integrity and compatibility of this patch release must be confirmed.

Please verify the following:

  1. SHA-256 checksum validity: Confirm that f86344275d1b194688dd330abf9f6f2344cd02872ffee035f2d1ea2fd60cf7f3 matches the official Gradle 9.2.1 distribution from gradle.org. You can cross-check against the Gradle release page or the official distribution listing.

  2. Testing and compatibility: Verify that Gradle 9.2.1 has been tested with this project's build and that there are no known breaking changes or issues introduced in this patch release.

  3. Scope verification: Confirm that no other Gradle-related configuration files (e.g., build.gradle, gradle.properties, settings.gradle) require updates to support Gradle 9.2.1.

Also applies to: 8-8

Comment @coderabbitai help to get the list of available commands and usage tips.

@reta reta added the Build Build Tasks/Gradle Plugin, groovy scripts, build tools, Javadoc enforcement. label Jan 8, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jan 8, 2026

❌ Gradle check result for 8aeb48f: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jan 9, 2026

❕ Gradle check result for 8aeb48f: UNSTABLE

Please review all flaky tests that succeeded after retry and create an issue if one does not already exist to track the flaky failure.

@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 9, 2026 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.19%. Comparing base (62eed1e) to head (8aeb48f).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files 
@@             Coverage Diff              @@
##               main   #20391      +/-   ##
============================================
- Coverage     73.23%   73.19%   -0.05%     
+ Complexity    71735    71686      -49     
============================================
  Files          5784     5784              
  Lines        328153   328196      +43     
  Branches      47270    47276       +6     
============================================
- Hits         240333   240208     -125     
- Misses        68498    68711     +213     
+ Partials      19322    19277      -45

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@cwperks cwperks merged commit 18b9df3 into opensearch-project:main Jan 9, 2026
45 of 47 checks passed
This was referenced Feb 18, 2026
Open
Open
Open
tanyabti pushed a commit to tanyabti/OpenSearch that referenced this pull request Feb 24, 2026
@andrross @tanyabti
Update to Gradle 9.2.1 (opensearch-project#20391)
6b5d70b 
Signed-off-by: Andrew Ross <andrross@amazon.com>
pradeep-L pushed a commit to pradeep-L/OpenSearch that referenced this pull request Apr 21, 2026
@andrross
Update to Gradle 9.2.1 (opensearch-project#20391)
da6f8d5 
Signed-off-by: Andrew Ross <andrross@amazon.com>
@opensearch-ci-bot opensearch-ci-bot mentioned this pull request Apr 27, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@reta reta reta approved these changes

@prudhvigodithi prudhvigodithi prudhvigodithi approved these changes

Assignees

No one assigned

Labels

Build Build Tasks/Gradle Plugin, groovy scripts, build tools, Javadoc enforcement. skip-changelog

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@andrross @reta @prudhvigodithi @cwperks